Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Gossip: CRDS Table implementation #6

Merged
merged 23 commits into from
Jul 24, 2023
Merged

Gossip: CRDS Table implementation #6

merged 23 commits into from
Jul 24, 2023

Conversation

0xNineteen
Copy link
Contributor

@0xNineteen 0xNineteen commented Jul 23, 2023 •
edited
Loading

  • init impl of the CRDS gossip table
  • also impl handle_push_message since its a straighforward insert

@0xNineteen
add pong verification
16c479b
@0xNineteen
modify verif sig + improve msg unpacking
8d07214
@0xNineteen
update random pubkey with seed
dbd3bf9
@0xNineteen
remove debug statement from tests
c50999f
@0xNineteen
add ping verification
70a6c01
@0xNineteen
change ip to localhost
c17356c
@0xNineteen
fix formatting
96518a3
@0xNineteen
init
3f12725
@0xNineteen
add crds insert test
c47c6bf
@0xNineteen
add node get fcn and tests
11afa51
@0xNineteen
Merge branch 'x19/ping-pong' of https://github.com/Syndica/sig into x…
4390961 
…19/crds-table
@0xNineteen
add overwrite logic
0ba7805
@0xNineteen
fix tests + formatting
e69d97b
@0xNineteen
add vote parsing logic
0dfaa3c
@0xNineteen
add the other data types to track
d3c8c58
@0xNineteen
Merge branch 'main' of https://github.com/Syndica/sig into x19/crds-t…
81f61db 
…able
@0xNineteen
fix
7f01d14
@0xNineteen
fix formatting
bd2d09b
@0xNineteen
handle push messages logic
0af8f42
@0xNineteen
fix test style
43689ef
@0xNineteen
add switch on error types
821af8e
0xNineteen
0xNineteen commented Jul 24, 2023
View reviewed changes
src/gossip/crds_table.zig Outdated Show resolved Hide resolved
@ultd ultd merged commit 789a518 into main Jul 24, 2023
@ultd ultd deleted the x19/crds-table branch July 24, 2023 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ultd ultd ultd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@0xNineteen @ultd